Skip to content

Adjustments for running hub in non-priv/non-systemd container #1718

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented May 19, 2025

  • Added hostname package dependency for rpm hub package
  • Added better check for systemd presence

Build Status

For UBI container images this package/command is not installed by default so require it as the installation scriptlets need it.

Ticket: ENT-12995
For example, an ubi9 container image contains systemctl command that will fail always because systemd is not started.

Ticket: ENT-12653
Changelog: none
@cf-bottom
Copy link

Thanks for submitting a PR! Maybe @craigcomstock can review this?

@craigcomstock craigcomstock marked this pull request as ready for review May 27, 2025 15:22
@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins please

@cf-bottom
Copy link

Copy link
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@@ -14,6 +14,10 @@ Url: https://cfengine.com
BuildRoot: %{_topdir}/%{name}-%{version}-%{release}-buildroot
Obsoletes: cfengine3 < @@VERSION@@, cf-community < @@VERSION@@
Requires: coreutils

# installation scriptlets need hostname command, not present in UBI container images
Requires: hostname
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we should/could do /bin/hostname here instead, but it's probably just the same in the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants